Skip to content

Conversation

@avasilev-sap
Copy link
Contributor

@avasilev-sap avasilev-sap commented Oct 7, 2025

#3698

Add system additional message as a state property of the config prompter in the ADP generator. For a system which requires authentication in order to determine the project type we need to first authenticate to the system. The system additional message contains the project type needed. The message can be set from the system prompt when the system does not require authentication or later in the password prompt when the login completes. In order to prevent from displaying the message twice we maintain a state pointing to the message in the prompter. Only one prompt - system or password could display that message.

Testing done:
Test scenario:

  1. Select cloud system which does not require auth and wait for the system to load. The project type appears beneath the system prompt.
  2. Select a cloud system which requires auth. The project type disappears.
  3. Enter credentials and press login.
    => When the login completes the project type appears beneath the system prompt.

Test scenario:

  1. Login to a cloud system.
  2. Press Start-over.
  3. Start again the generator.
    => The project type appears beneath the password prompt.

@changeset-bot
Copy link

changeset-bot bot commented Oct 7, 2025

🦋 Changeset detected

Latest commit: acf883b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sap-ux/generator-adp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@avasilev-sap avasilev-sap self-assigned this Oct 7, 2025
@avasilev-sap avasilev-sap added the generator-adp @sap-ux/generator-adp label Oct 7, 2025
@avasilev-sap avasilev-sap changed the title Project name appears twice under system and password prompts when cloud system is selected. fix: Project name appears twice under system and password prompts when cloud system is selected. Oct 7, 2025
@avasilev-sap avasilev-sap marked this pull request as ready for review October 7, 2025 13:36
@avasilev-sap avasilev-sap changed the title fix: Project name appears twice under system and password prompts when cloud system is selected. fix: Project type appears twice under system and password prompts when cloud system is selected. Oct 8, 2025
Copy link
Contributor

@mmilko01 mmilko01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changeset OK
Changes look good
Excellent test coverage
Did not test manually

@nikmace nikmace self-requested a review October 8, 2025 11:34
Copy link
Contributor

@nikmace nikmace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good
Changeset is good
Test coverage is good
Did not test manually

@sonarqubecloud
Copy link

@avasilev-sap avasilev-sap merged commit 3d63709 into main Oct 14, 2025
16 checks passed
@avasilev-sap avasilev-sap deleted the fix/3698/project-type branch October 14, 2025 17:31
devinea added a commit that referenced this pull request Oct 15, 2025
…chart_on_list_v4

* origin/main:
  chore: apply latest changesets
  feat(fpm-writer): table building block custom columns (#3665)
  chore: apply latest changesets
  fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706)
  chore: apply latest changesets
  fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times.  (#3683)
  chore: apply latest changesets
  update axios version (#3732)
  chore: apply latest changesets
  fix(fiori-annotation-api): wrong origin range (#3729)
  chore: apply latest changesets
  TBI - Replace service key (UAA/OAuth2) based connections with re-entrance tickets (#3693)
  chore: apply latest changesets
  fix: smartlinks target prompt (#3643)
  chore: apply latest changesets
  feat: Add ADP Generator Cloud Foundry prompting code (#3518)
devinea added a commit that referenced this pull request Oct 16, 2025
…r_RAG_code_gen

* origin/main: (26 commits)
  chore: apply latest changesets
  feat(odata-service-inquirer): 35947 default table type (#3725)
  chore: apply latest changesets
  Fix/35934/incorrect entity list rich text editor (#3724)
  fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735)
  chore: apply latest changesets
  fix: vocabulary update (#3733)
  chore: apply latest changesets
  feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717)
  fix: schedule ui integration tests on every workday (#3620)
  chore: apply latest changesets
  feat(fpm-writer): table building block custom columns (#3665)
  chore: apply latest changesets
  fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706)
  chore: apply latest changesets
  fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times.  (#3683)
  chore: apply latest changesets
  update axios version (#3732)
  chore: apply latest changesets
  fix(fiori-annotation-api): wrong origin range (#3729)
  ...
devinea added a commit that referenced this pull request Oct 16, 2025
* origin/main: (286 commits)
  chore: apply latest changesets
  feat(odata-service-inquirer): 35947 default table type (#3725)
  chore: apply latest changesets
  Fix/35934/incorrect entity list rich text editor (#3724)
  fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735)
  chore: apply latest changesets
  fix: vocabulary update (#3733)
  chore: apply latest changesets
  feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717)
  fix: schedule ui integration tests on every workday (#3620)
  chore: apply latest changesets
  feat(fpm-writer): table building block custom columns (#3665)
  chore: apply latest changesets
  fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706)
  chore: apply latest changesets
  fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times.  (#3683)
  chore: apply latest changesets
  update axios version (#3732)
  chore: apply latest changesets
  fix(fiori-annotation-api): wrong origin range (#3729)
  ...
tom-sap added a commit that referenced this pull request Oct 16, 2025
* main: (61 commits)
  chore: apply latest changesets
  feat(odata-service-inquirer): 35947 default table type (#3725)
  chore: apply latest changesets
  Fix/35934/incorrect entity list rich text editor (#3724)
  fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735)
  chore: apply latest changesets
  fix: vocabulary update (#3733)
  chore: apply latest changesets
  feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717)
  fix: schedule ui integration tests on every workday (#3620)
  chore: apply latest changesets
  feat(fpm-writer): table building block custom columns (#3665)
  chore: apply latest changesets
  fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706)
  chore: apply latest changesets
  fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times.  (#3683)
  chore: apply latest changesets
  update axios version (#3732)
  chore: apply latest changesets
  fix(fiori-annotation-api): wrong origin range (#3729)
  ...
eouin added a commit that referenced this pull request Oct 22, 2025
…ter/building-block-custom-filter

* 'main' of github.com:SAP/open-ux-tools:
  chore: apply latest changesets
  feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717)
  fix: schedule ui integration tests on every workday (#3620)
  chore: apply latest changesets
  feat(fpm-writer): table building block custom columns (#3665)
  chore: apply latest changesets
  fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706)
  chore: apply latest changesets
  fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times.  (#3683)
  chore: apply latest changesets
  update axios version (#3732)
  chore: apply latest changesets
  fix(fiori-annotation-api): wrong origin range (#3729)
  chore: apply latest changesets
  TBI - Replace service key (UAA/OAuth2) based connections with re-entrance tickets (#3693)
  chore: apply latest changesets
  fix: smartlinks target prompt (#3643)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

generator-adp @sap-ux/generator-adp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants